Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #77 approve bitbucket PRs when configured. #541

Closed
wants to merge 1 commit into from
Closed

Fix #77 approve bitbucket PRs when configured. #541

wants to merge 1 commit into from

Conversation

nfalco79
Copy link

@nfalco79 nfalco79 commented Feb 1, 2022

This feature requires a write access configured in OAuth consumers.
Add a new configuration parameter under ALMSettings/Bitbucket to enable or not this behavior (default false). When enabled and sonar analysis a pull request branch the logged user will approve/disapprove current PR based on quality gate result.

Before spend time to implement test case please let me know if you are yet interested on this (#77) feature

immagine
In picture seems I can not change how "bitbucket" sub category appears, adding a description for example. It looks like something that depends on sonarqube server.

This feature requires a write access configured in OAuth consumers.
Add a new configuration parameter under ALMSettings/Bitbucket to enable or not this behavior (default false). When enabled and sonar analysis a pull request branch the logged user will approve/disapprove current PR based on quality gate result.
@mc1arke
Copy link
Owner

mc1arke commented Feb 1, 2022

You can't add options on the MR/PR configuration screens given the way they're built. I'm also not a fan of adding configuration on other screens as it makes it difficult for users to find all the configuration options they're supposed to set. Is approvals something that Sonarqube offers in its commercial offerings?

@nfalco79
Copy link
Author

nfalco79 commented Feb 1, 2022

Do you prefer a dedicated page? But this page should refer only to bitbucket action

Is approvals something that Sonarqube offers in its commercial offerings?

No at least from documentation and also because their setup does not describe Write right for PullRequest on Oauth consumers, this means it can not perform aprrove action like this

@nfalco79
Copy link
Author

nfalco79 commented Feb 1, 2022

I did there because is strctly resrticted to bitbucket ALM integration behavior. I see you removed a lot of custom properties relying on branch default. Actually there is only one in general settings (category).

@nfalco79
Copy link
Author

Closed in favor of this plugin https://github.com/nfalco79/sonar-bitbucket-cloud to approve PullRequests

@nfalco79 nfalco79 closed this Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants